[18.0][IMP] rma*: Improved portal and list views#604
Conversation
|
Hi @sbejaoui, @chienandalu, @pedrobaeza, |
OCA-git-bot
added
mod:rma
|
Can you put some screenshots? |
Added to the PR description. |
carlos-lopez-tecnativa
left a comment
carlos-lopez-tecnativa
left a comment
There was a problem hiding this comment.
LGTM, thanks for these improvements. Could you add an icon to the RMA section, like the other sections in the portal?
The module icon can be displayed in the portal.
pedrobaeza
left a comment
pedrobaeza
left a comment
There was a problem hiding this comment.
Using AI having the existing icons as guidance, the proposal is:
victoralmau
force-pushed
the
18.0-imp-misc-portal
branch
from
6d7cb55 to
77f43c0
Compare
|
Currently, there is no “Origin” column in the RMA list, but I think it might be useful to have one, especially when creating RMAs from another RMA. |
|
Put the "Origin" column for both RMA or sale, and make it clickable. |
victoralmau
force-pushed
the
18.0-imp-misc-portal
branch
from
77f43c0 to
7536032
Compare
|
I think this can be packed a bit more than 10 commits. You can join all the column improvements together, even if from different modules, as they are related in the logical block. |
victoralmau
force-pushed
the
18.0-imp-misc-portal
branch
from
7536032 to
7cc8217
Compare
4 participants
Changes done:
rma+rma_sale: Move Requested operation (operation_id) block to rma pagerma: Add name to th/td elements to rma list portal pagerma+rma_sale: Add Origin (order_id or origin) column to rma page portal listrma_lot: Add Serial Number/Lot column to rma page portal listrma_reason: Add Reason column to rma page portal listrma_sale: Display the Sale Order field as a link on the RMA portal formrma: Improvements to the RMA section of /my (text + icon)Before
After
Please @pedrobaeza and @carlos-lopez-tecnativa can you review it?
@Tecnativa